-
Notifications
You must be signed in to change notification settings - Fork 61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding a service to set the image topic. #17
base: master
Are you sure you want to change the base?
Conversation
Sorry for all the tiny commits. I'll squash them if I can get past Jenkins. |
ping @dirk-thomas |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am sorry for the late reply. I will hopefully get this released as soon as you had a chance to address the feedback. Thanks.
Numerous of my previous comments have been marked as resolved but not actually been addressed? |
…ot fix the problem if you rosrun multiple image views
Sorry, I'm not sure how I missed so many comments |
This PR is close to being 3 years untouched. I also think is overrated to control a GUI by a service. We have a gui, when we don't want command line. @mabelzhang I suggest you close this PR |
No description provided.